Skip to content

Conversation

@samad-yar-khan
Copy link
Contributor

@samad-yar-khan samad-yar-khan commented Mar 25, 2022

Issues

Mentioned in the issue #72

Proposed changes (including videos or screenshots)

  • New Reusable Gitlab Components Kit

  • Data fetching method in cms (can be re-used when switching from strapi) to provide a better experience for community builders. All data fetching for the gitalb kit for a repository will be done by a a single gitlabKit() function.

  • Introduce Single <Gitlab ... > tag which can be used to fetch different components of the kit depending on props/parameters. <Gitlab type={'members'} gitlabData={props.gitlabData} />

  • Introduce single gitlabKitData(project_id ,[..needs]) function which can fetch relevant data based on the components we wish to use on a particular page.

  • New Gitlab Merge Request Component

    merge-example

  • New Gitlab Project Members Component

members-example

  • New Gitlab Project Overview Component

overview

  • New Gitlab Project Issues Component

issues-example

2022-03-27.16-56-32.mp4
  • New Gitlab Page to showcase the components kit which can be accessed at /gitlab
2022-03-27.14-51-24.mp4

@samad-yar-khan samad-yar-khan changed the title [WIP] [NEW] Gitlab Component Kit [NEW] Gitlab Component Kit Mar 27, 2022
@lgtm-com
Copy link

lgtm-com bot commented Mar 27, 2022

This pull request introduces 1 alert when merging acf335c into 0930180 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@samad-yar-khan
Copy link
Contributor Author

@Sing-Li I have added all the necessary to the Gitlab Components Kit , added documentation and have also created a page to showcase the component. Let me if know if there are any changes 😅

@lgtm-com
Copy link

lgtm-com bot commented Apr 3, 2022

This pull request introduces 1 alert when merging 03e351d into 6541e17 - view on LGTM.com

new alerts:

  • 1 for Useless assignment to local variable

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant